Skip to content

Refactor/all - #28

Merged
Jamkris merged 25 commits into
mainfrom
refactor/all
Apr 8, 2026
Merged

Refactor/all#28
Jamkris merged 25 commits into
mainfrom
refactor/all

Conversation

@Jamkris

@Jamkris Jamkris commented Apr 8, 2026

Copy link
Copy Markdown
Owner

No description provided.

Jamkris added 23 commits April 8, 2026 09:32
…revent namespace conflicts and remove stale shims during setup

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request rebrands the project from 'Claude Code' to 'Gemini Code,' updating paths, environment variables, and documentation across the repository. It refactors CI validation and hook scripts to use new shared utility libraries, validator.js and hook-utils.js, and adds unit tests for session management. Feedback includes removing an unnecessary shebang from the library module and improving the robustness of the new utility functions by handling synchronous hooks, using buffers for stdin, and improving line-ending parsing.

Comment thread scripts/lib/validator.js Outdated
Comment thread scripts/lib/hook-utils.js
Comment thread scripts/lib/hook-utils.js
Comment thread scripts/lib/validator.js Outdated
@Jamkris
Jamkris merged commit 9281e40 into main Apr 8, 2026
7 checks passed
@Jamkris Jamkris added the 🔨 Refactor Code refactoring with no behavior change label Apr 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 Refactor Code refactoring with no behavior change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant